Skip to content

[BI-2778] Deprecate additionalinfo usage for level and use obs unit lvl#459

Merged
jloux-brapi merged 3 commits intodevelopfrom
feature/BI-2778
Mar 10, 2026
Merged

[BI-2778] Deprecate additionalinfo usage for level and use obs unit lvl#459
jloux-brapi merged 3 commits intodevelopfrom
feature/BI-2778

Conversation

@HMS17
Copy link
Contributor

@HMS17 HMS17 commented Mar 6, 2026

Description

Story: BI-2778 - Deprecate additionalinfo usage for level and use obs unit level

Changes to ensure front end display of obs lvl is capitalized.

Dependencies

bi-api: feature/BI-2778

Testing

see bi-api

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth
  • I have run SiteImprove on pages impacted by changes

@HMS17 HMS17 marked this pull request as ready for review March 9, 2026 23:08
@HMS17 HMS17 requested review from a team and jloux-brapi and removed request for a team March 9, 2026 23:16
Comment on lines +99 to +103
computed: {
StringFormatters() {
return StringFormatters
}
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that I'm seeing this for a second time, I'm wondering if there's a more elegant way to do this without this same boilerplate code.

If we can create some sort of global obj/utility that can be used by any component I think we could do that, but I do know it's more work.

For now, I think what you have is fine, but we should start to think about this if the use of StringFormatters continues to get larger.

@jloux-brapi jloux-brapi merged commit 4b937fa into develop Mar 10, 2026
1 check passed
@jloux-brapi jloux-brapi deleted the feature/BI-2778 branch March 10, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants